From: Florian Müllner <fmuellner@gnome.org>
Date: Wed, 22 Jul 2020 00:05:48 +0000 (+0200)
Subject: layout: Only show ripple animation when overview was toggled
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~684^2^2~91
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=71f55643b275c6b8761f6ef1e6a3150fdfdd5c28;p=gnome-shell.git

layout: Only show ripple animation when overview was toggled

On X11, clients can grab keyboard on pointer (for example for popup
menus), and as a result the pushModal() call when opening the overview
fails.

However when the hot corner was used to toggle the overview, we still
show the ripple animation in that case, which is confusing as the action
did not actually happen.

Fix this by only showing the ripples if the overview is animating after
calling toggle(), as that should be a reliable indication of whether
the call was successful.

https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/3005
---

diff --git a/js/ui/layout.js b/js/ui/layout.js
index d37c76940..b082ec80b 100644
--- a/js/ui/layout.js
+++ b/js/ui/layout.js
@@ -1228,8 +1228,9 @@ class HotCorner extends Clutter.Actor {
             return;
 
         if (Main.overview.shouldToggleByCornerOrButton()) {
-            this._ripples.playAnimation(this._x, this._y);
             Main.overview.toggle();
+            if (Main.overview.animationInProgress)
+                this._ripples.playAnimation(this._x, this._y);
         }
     }